Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gossipd: don't discard node_announcements with old timestamps. #3262

Merged

Conversation

rustyrussell
Copy link
Contributor

It really, really doesn't matter. But we were dramatically reducing
our view of the network:

In my gossip_store (mainnet):
channel_announcement: 30349
channel_update: 55119
node_announcment: 1783

Fixes: #3194

@rustyrussell rustyrussell added this to the 0.7.4 milestone Nov 15, 2019
It really, really doesn't matter.  But we were dramatically reducing
our view of the network:

In my gossip_store (mainnet):
  channel_announcement: 30349
  channel_update: 55119
  node_announcment: 1783

Changelog-Fixed: No longer discard most node_announcements (fixes ElementsProject#3194)
Signed-off-by: Rusty Russell <[email protected]>
@cdecker
Copy link
Member

cdecker commented Nov 15, 2019

ACK 52c51e9

@cdecker cdecker merged commit fdd69af into ElementsProject:master Nov 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Some aliases no longer visible in listnodes
2 participants